Skip to content
This repository was archived by the owner on Apr 26, 2024. It is now read-only.

Dev refactor savelogs#126

Merged
oracle58 merged 34 commits intoHelixNetwork:devfrom
dnck:dev-refactor-savelogs
Sep 5, 2019
Merged

Dev refactor savelogs#126
oracle58 merged 34 commits intoHelixNetwork:devfrom
dnck:dev-refactor-savelogs

Conversation

@dnck
Copy link
Copy Markdown
Contributor

@dnck dnck commented Sep 5, 2019

This is the refactor of the saveLogs method suggested in Issue #124.

dnck added 30 commits March 27, 2019 13:41
Pull in of changes timestamp validation, gitignore, writing to snapshot fixed, snapshot time, timestamp in ms, spend addresses on loadup false
@oracle58
Copy link
Copy Markdown
Contributor

oracle58 commented Sep 5, 2019

Codecov Report

Merging #126 into dev will increase coverage by <.01%.
The diff coverage is 0%.

Impacted file tree graph

@@             Coverage Diff              @@
##                dev     #126      +/-   ##
============================================
+ Coverage     46.35%   46.36%   +<.01%     
- Complexity     1225     1227       +2     
============================================
  Files           160      160              
  Lines          7623     7626       +3     
  Branches        934      934              
============================================
+ Hits           3534     3536       +2     
- Misses         3715     3718       +3     
+ Partials        374      372       -2
Impacted Files Coverage Δ Complexity Δ
...rc/main/java/net/helix/hlx/utils/HelixIOUtils.java 17.39% <0%> (-2.61%) 2 <0> (ø)
src/main/java/net/helix/hlx/network/Node.java 39.39% <0%> (+0.23%) 46% <0%> (+1%) ⬆️
...ilestone/impl/LatestSolidMilestoneTrackerImpl.java 47.82% <0%> (+1.44%) 10% <0%> (+1%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0ebf932...71f82e3. Read the comment docs.

@oracle58 oracle58 merged commit 5561107 into HelixNetwork:dev Sep 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants